Skip to content

Align Pane Chat skill contract - #355

Merged
parsakhaz merged 1 commit into
mainfrom
issue-354-contract-sync-discussion
Jul 22, 2026
Merged

Align Pane Chat skill contract#355
parsakhaz merged 1 commit into
mainfrom
issue-354-contract-sync-discussion

Conversation

@parsakhaz

Copy link
Copy Markdown
Member

Summary

  • Make Pane Chat's generated guide and pane-orchestrator define contract precedence instead of duplicating upstream lifecycle policy.
  • Add required raw fallback coverage for Codex/Claude gh-address-comments, PR-test metadata, and Claude review support.
  • Fail raw fallback sync when lifecycle-critical required downloads fail, even if stale cached files already exist.
  • Update Pane Chat README docs to describe runtime/generated/upstream/downstream responsibility boundaries.

Closes #354

Validation

  • pnpm --filter main exec vitest run src/services/skillCacheManager.test.ts
  • pnpm --filter main typecheck
  • pnpm --filter main lint (passes with existing warnings outside this change)
  • git diff --check origin/main..HEAD

Notes

No visual asset is included because this change is limited to generated text, fallback cache coverage, and tests; a diagram would not add useful review evidence.

@parsakhaz
parsakhaz merged commit 277f95a into main Jul 22, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: align Pane Chat contract with proactive skills lifecycle

1 participant